Skip to content

.Net: Upgraded Azure OpenAI SDK version.#10487

Closed
shethaadit wants to merge 2 commits intomicrosoft:mainfrom
shethaadit:shethaadit/FixBug10484
Closed

.Net: Upgraded Azure OpenAI SDK version.#10487
shethaadit wants to merge 2 commits intomicrosoft:mainfrom
shethaadit:shethaadit/FixBug10484

Conversation

@shethaadit
Copy link
Copy Markdown
Contributor

@shethaadit shethaadit commented Feb 11, 2025 •
edited
Loading

Description

Upgraded Azure OpenAI SDK version.

Fixes #10484

@shethaadit shethaadit requested a review from a team as a code owner February 11, 2025 17:59
@markwallace-microsoft markwallace-microsoft added the .NET Issue or Pull requests regarding .NET code label Feb 11, 2025
@github-actions github-actions Bot changed the title Upgraded Azure OpenAI SDK version. .Net: Upgraded Azure OpenAI SDK version. Feb 11, 2025
@shethaadit
Copy link
Copy Markdown
Contributor Author

shethaadit commented Feb 11, 2025

Hi @markwallace-microsoft / @rogerbarreto, could you please approve the build run? I would like to see if the build gets passed since my local set up is broken.

@dmytrostruk
Copy link
Copy Markdown
Member

dmytrostruk commented Feb 11, 2025

@shethaadit I approved build run, it looks like there are some errors, here are a couple of them:

Error: /home/runner/work/semantic-kernel/semantic-kernel/dotnet/src/Agents/OpenAI/Internal/AssistantThreadActions.cs(764,69): error CS0103: The name 'RunStepType' does not exist in the current context [/home/runner/work/semantic-kernel/semantic-kernel/dotnet/src/Agents/OpenAI/Agents.OpenAI.csproj::TargetFramework=netstandard2.0]
Error: /home/runner/work/semantic-kernel/semantic-kernel/dotnet/src/Agents/OpenAI/Internal/AssistantThreadActions.cs(770,98): error CS1061: 'RunStepToolCall' does not contain a definition for 'ToolCallId' and no accessible extension method 'ToolCallId' accepting a first argument of type 'RunStepToolCall' could be found (are you missing a using directive or an assembly reference?) [/home/runner/work/semantic-kernel/semantic-kernel/dotnet/src/Agents/OpenAI/Agents.OpenAI.csproj::TargetFramework=netstandard2.0]

@dmytrostruk
Copy link
Copy Markdown
Member

dmytrostruk commented Feb 11, 2025

@shethaadit Please see this commit 6f0b6f0, it contains required changes to align with introduced breaking changes from latest OpenAI SDK release.

@shethaadit
Copy link
Copy Markdown
Contributor Author

shethaadit commented Feb 11, 2025

@shethaadit Please see this commit 6f0b6f0, it contains required changes to align with introduced breaking changes from latest OpenAI SDK release.

Thank you @dmytrostruk for the commit.

@dmytrostruk dmytrostruk mentioned this pull request Feb 12, 2025
Merged
@rogerbarreto
Copy link
Copy Markdown
Member

rogerbarreto commented Feb 12, 2025

@shethaadit Thanks for the proposed PR.

As @dmytrostruk also noted I have create another one including some already missing features we need to consider during the upgrade:

I'm closing this PR as the above covers it. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

.NET Issue or Pull requests regarding .NET code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

.Net: Update Both (OpenAI + AzureOpenAI) Connectors to use 2.2.0-beta.1 version.

4 participants

@shethaadit @dmytrostruk @rogerbarreto @markwallace-microsoft